Skip to content

fix(ci): tolerate frontend test artifact errors - #104360

Merged
trunk-io[bot] merged 1 commit into
masterfrom
posthog/fix-frontend-artifact-upload
Sep 25, 2026
Merged

fix(ci): tolerate frontend test artifact errors#104360
trunk-io[bot] merged 1 commit into
masterfrom
posthog/fix-frontend-artifact-upload

Conversation

@pauldambra

@pauldambra pauldambra commented Sep 22, 2026

Copy link
Copy Markdown
Member

Problem

A passing frontend test shard can fail a merge-queue batch when GitHub rejects its test-result artifact during finalization (evidence).

Why: Test validity must not depend on the artifact service.

Changes

  • The artifact upload now reports failure without changing a passing Jest shard verdict.
  • The quarantine gate still reads local JUnit results, so known flakes keep their existing handling.

Before:

flowchart LR
    Jest[Jest result] --> Artifact[Test-result artifact]
    Artifact --> Quarantine[Quarantine gate]
    Artifact --> Failure[Failed shard]
    Quarantine --> Gate[Required test gate]
    classDef phBlue fill:#1d4aff,stroke:#1d4aff,color:#fff;
    classDef phRed fill:#f54e00,stroke:#f54e00,color:#fff;
    classDef phYellow fill:#f9bd2b,stroke:#f9bd2b,color:#000;
    class Jest phYellow;
    class Artifact phRed;
    class Quarantine,Gate phBlue;
    class Failure phRed;
Loading

After:

flowchart LR
    Jest[Jest result] --> Artifact[Test-result artifact]
    Artifact --> Quarantine[Quarantine gate]
    Artifact --> Warning[Upload warning]
    Quarantine --> Gate[Required test gate]
    classDef phBlue fill:#1d4aff,stroke:#1d4aff,color:#fff;
    classDef phRed fill:#f54e00,stroke:#f54e00,color:#fff;
    classDef phYellow fill:#f9bd2b,stroke:#f9bd2b,color:#000;
    class Jest phYellow;
    class Artifact,Warning phRed;
    class Quarantine,Gate phBlue;
Loading

How did you test this code?

  • bin/hogli lint:workflows
  • actionlint on this workflow, excluding its existing unsupported native parallel: syntax
  • bin/hogli ci:preflight

Automatic notifications

  • Publish to changelog?

Docs update

No. The test and quarantine behavior stays the same.

🤖 Agent context

Autonomy: Human-driven (agent-assisted)

Agent: PostHog Slack app, GPT-5

  • Skills: /debugging-ci-failures, /authoring-ci-workflows, /writing-code-comments, /writing-pr-descriptions, and /reviewing-with-coderabbit.
  • CodeRabbit CLI was unavailable, so this PR opened without a local CodeRabbit pass.
  • No matching open issue was found.

Created with PostHog from a Slack thread

@pauldambra
fix(ci): tolerate frontend test artifact errors
8ca6b6a 
Generated-By: PostHog Desktop
Task-Id: 4954a5cb-231a-49d5-8920-d611aa7ad405
@pauldambra pauldambra self-assigned this Sep 22, 2026
@pauldambra
pauldambra requested a review from a team September 22, 2026 11:51
@trunk-io

trunk-io Bot commented Sep 22, 2026 •
edited
Loading

Copy link
Copy Markdown

😎 Merged successfully - details.

@coderabbitai

coderabbitai Bot commented Sep 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: PostHog/posthog/.coderabbit.yaml

Review profile: QUIET

Plan: Enterprise

Run ID: f0d23940-7948-4736-90b5-14cf1cda263a

📥 Commits

Reviewing files that changed from the base of the PR and between d2c5868 and 8ca6b6a.

📒 Files selected for processing (1)
  • .github/workflows/ci-frontend.yml

Included review availability: Your plan provides up to 12 included reviews per hour; 10 remain after this review.

📝 Walkthrough

Walkthrough

The Jest test-results artifact upload step in the frontend CI workflow now continues on error. Artifact-service failures no longer determine the Jest job verdict.

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 8ca6b

The workflow tolerates artifact upload failures while preserving Jest shard results; no merge-blocking risk remains.

🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The description is complete and matches the required structure. It explains the problem, changes, testing, documentation impact, agent context, and workflow topology. The required Release status selec…
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Sep 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

🤖 CI report

✅ Trunk lane — non-backend lane

This PR is assigned to the non-backend lane. It does not run backend Python tests and may merge in parallel with PRs in other lanes.

✅ Bundle size — 🟢 -872.0 KiB (-1.2%)

Uncompressed size of every built .js bundle, compared against the base branch.

Total: 68.37 MiB · 🟢 -872.0 KiB (-1.2%)

File Size Δ vs base
render-query/src/render-query/render-query.js 20.16 MiB 🟢 -1.72 MiB (-7.9%)
posthog-app/_parent/node_modules/.pnpm/v86@0.5.458_patch_hash=567937f0364248ae4346886b1da715b6e5a29dd6835c33b66f3264a6bb95d37f/node_modules/v86/build/libv86.js 349.1 KiB 🔺 +349.1 KiB (new)
exporter/src/exporter/scenes/ExporterNotebookScene.js 3.67 MiB 🔺 +61.2 KiB (+1.7%)
posthog-app/_parent/products/tracing/frontend/TracingScene.js 176.9 KiB 🔺 +57.9 KiB (+48.6%)
posthog-app/_parent/products/data_warehouse/frontend/scenes/DataOpsScene/DataWarehouseScene.js 49.0 KiB 🔺 +49.0 KiB (new)
posthog-app/_parent/products/workflows/frontend/Broadcasts/BroadcastScene.js 40.0 KiB 🔺 +40.0 KiB (new)
posthog-app/src/scenes/models/ModelsScene.js removed 🟢 -33.8 KiB (-100.0%)
posthog-app/_parent/products/data_modeling/frontend/ModelsScene.js 33.7 KiB 🔺 +33.7 KiB (new)
posthog-app/_parent/products/data_modeling/frontend/nodeDetail/NodeDetailScene.js 25.2 KiB 🔺 +25.2 KiB (new)
posthog-app/src/scenes/models/NodeDetailScene.js removed 🟢 -25.2 KiB (-100.0%)
posthog-app/_parent/products/experiments/frontend/scenes/ExperimentsScene.js 62.6 KiB 🔺 +23.9 KiB (+61.9%)
posthog-app/_parent/products/ml_inference/frontend/DecisionPlaygroundScene.js 16.1 KiB 🔺 +16.1 KiB (new)
exporter/_parent/products/posthog_ai/frontend/scenes/TaskTracker/TaskTracker.js 14.6 KiB 🔺 +14.6 KiB (new)
posthog-app/src/scenes/AuthenticatedShell.js 263.3 KiB 🔺 +12.5 KiB (+5.0%)
exporter/_parent/products/error_tracking/frontend/posthogAi/ErrorTrackingWidget.js 11.8 KiB 🔺 +11.8 KiB (new)
posthog-app/_parent/products/signals/frontend/inbox/InboxScene.js 465.3 KiB 🔺 +11.1 KiB (+2.4%)
exporter/_parent/products/posthog_ai/frontend/scenes/TaskTracker/components/SidePanelRunnerImpl.js 9.7 KiB 🔺 +9.7 KiB (new)
posthog-app/src/layout/navigation-3000/sidepanel/panels/support/SidePanelSupport.js 9.7 KiB 🔺 +9.7 KiB (new)
exporter/_parent/products/posthog_ai/frontend/components/tool/builtinToolRenderers.js 9.4 KiB 🔺 +9.4 KiB (new)
posthog-app/src/lib/components/CodeSnippet/HighlightedCodeLine.js 8.9 KiB 🔺 +8.9 KiB (new)
exporter/src/lib/components/CodeSnippet/HighlightedCodeLine.js 8.7 KiB 🔺 +8.7 KiB (new)
exporter/src/queries/schema.js 1.22 MiB 🔺 +7.8 KiB (+0.6%)
posthog-app/src/queries/schema.js 1.22 MiB 🔺 +7.8 KiB (+0.6%)
posthog-app/src/scenes/data-pipelines/batch-exports/BatchExportScene.js 66.3 KiB 🟢 -6.4 KiB (-8.8%)
exporter/_parent/products/cdp/frontend/HogFunctionPermissionPreview.js 5.4 KiB 🔺 +5.4 KiB (new)
posthog-app/src/scenes/terminal/TerminalScene.js 5.0 KiB 🔺 +5.0 KiB (new)
exporter/_parent/products/posthog_ai/frontend/components/tool/widgets/QueryWidget.js 4.9 KiB 🔺 +4.9 KiB (new)
exporter/_parent/products/posthog_ai/frontend/components/tool/widgets/CreateInsightWidget.js 4.9 KiB 🔺 +4.9 KiB (new)
posthog-app/_parent/products/data_catalog/frontend/DataCatalogMetricScene.js 31.0 KiB 🔺 +4.8 KiB (+18.2%)
exporter/_parent/products/posthog_ai/frontend/components/tool/widgets/SearchSessionRecordingsWidget.js 4.8 KiB 🔺 +4.8 KiB (new)
posthog-app/_parent/products/workflows/frontend/Workflows/hogflows/filters/HogFlowTaxonomicFilters.js 4.8 KiB 🔺 +4.8 KiB (new)
posthog-app/_parent/products/data_warehouse/DataWarehouseScene.js removed 🟢 -4.6 KiB (-100.0%)
posthog-app/src/scenes/data-warehouse/DataWarehouseScene.js removed 🟢 -4.6 KiB (-100.0%)
exporter/_parent/products/alerts/frontend/views/EditAlertModal.js 108.4 KiB 🔺 +4.5 KiB (+4.3%)
exporter/_parent/products/tasks/frontend/components/TaskRepositoryPicker/CyclotronJobInputTaskRepository.js 1.7 KiB 🟢 -4.4 KiB (-71.8%)
posthog-app/_parent/products/workflows/frontend/WorkflowsScene.js 102.4 KiB 🔺 +4.4 KiB (+4.5%)
posthog-app/src/scenes/project-files/ProjectFilesScene.js 4.4 KiB 🔺 +4.4 KiB (new)
exporter/_parent/products/posthog_ai/frontend/components/tool/posthogCodeToolRenderers.js 4.3 KiB 🔺 +4.3 KiB (new)
posthog-app/src/scenes/feature-flags/FeatureFlag.js 113.0 KiB 🔺 +3.8 KiB (+3.4%)
exporter/_parent/products/workflows/frontend/Workflows/Workflow.js 121.6 KiB 🟢 -3.7 KiB (-3.0%)
exporter/_parent/products/posthog_ai/frontend/components/tool/EditDiffRenderer.js 3.7 KiB 🔺 +3.7 KiB (new)
exporter/_parent/products/workflows/frontend/Workflows/hogflows/filters/HogFlowTaxonomicFilters.js 3.4 KiB 🔺 +3.4 KiB (new)
posthog-app/src/scenes/terminal/TerminalPanel.js 3.3 KiB 🔺 +3.3 KiB (new)
exporter/_parent/products/logs/frontend/agentTools/LogsQueryToolWidget.js 3.0 KiB 🔺 +3.0 KiB (new)
posthog-app/src/scenes/hog-functions/HogFunctionScene.js 56.5 KiB 🔺 +3.0 KiB (+5.7%)
posthog-app/_parent/products/engineering_analytics/frontend/scenes/EngineeringAnalyticsTeamScene.js 15.0 KiB 🔺 +2.6 KiB (+21.3%)
posthog-app/_parent/products/mcp_analytics/frontend/MCPAnalyticsScene.js 175.0 KiB 🟢 -2.4 KiB (-1.4%)
posthog-app/_parent/products/ai_observability/frontend/AIObservabilityScene.js 186.7 KiB 🔺 +2.4 KiB (+1.3%)
posthog-app/src/scenes/marketing-analytics/MarketingAnalyticsScene.js 137.2 KiB 🔺 +2.1 KiB (+1.6%)
posthog-app/_parent/products/web_analytics/frontend/heatmaps/scenes/heatmaps/HeatmapsScene.js 16.3 KiB 🔺 +2.1 KiB (+14.7%)
exporter/_parent/products/posthog_ai/frontend/components/tool/widgets/CreateNotebookWidget.js 2.0 KiB 🔺 +2.0 KiB (new)
exporter/_parent/products/posthog_ai/frontend/components/tool/widgets/UpsertDashboardWidget.js 1.9 KiB 🔺 +1.9 KiB (new)
exporter/_parent/products/posthog_ai/frontend/components/QuestionRenderer.js 1.8 KiB 🔺 +1.8 KiB (new)
posthog-app/_parent/products/conversations/frontend/scenes/ticket/SupportTicketScene.js 66.6 KiB 🔺 +1.8 KiB (+2.8%)
posthog-app/_parent/products/mcp_analytics/frontend/MCPAnalyticsToolDetail.js 29.6 KiB 🔺 +1.7 KiB (+6.1%)
posthog-app/_parent/products/groups/frontend/pages/Group.js 33.9 KiB 🔺 +1.7 KiB (+5.2%)
posthog-app/src/scenes/notebooks/NotebookScene.js 30.9 KiB 🔺 +1.7 KiB (+5.7%)
posthog-app/_parent/products/persons/frontend/pages/PersonScene.js 43.3 KiB 🔺 +1.7 KiB (+4.0%)
posthog-app/src/scenes/notebooks/NotebookPanel/NotebookPanel.js 25.3 KiB 🔺 +1.7 KiB (+7.1%)
posthog-app/src/scenes/notebooks/NotebookCanvasScene.js 23.9 KiB 🔺 +1.7 KiB (+7.5%)
posthog-app/_parent/products/business_knowledge/frontend/scenes/KnowledgeSourceScene.js 14.1 KiB 🔺 +1.6 KiB (+13.1%)
posthog-app/_parent/products/conversations/frontend/components/ComposeTicket/ComposeTicketModal.js 1.4 KiB 🔺 +1.4 KiB (new)
exporter/src/lib/components/ActivityLog/describers.js 162.0 KiB 🟢 -1.4 KiB (-0.9%)
posthog-app/_parent/products/logs/frontend/LogsScene.js 48.5 KiB 🔺 +1.4 KiB (+2.9%)
posthog-app/_parent/products/error_tracking/frontend/scenes/ErrorTrackingIssueScene/ErrorTrackingIssueScene.js 92.6 KiB 🔺 +1.3 KiB (+1.5%)
posthog-app/_parent/products/replay_vision/frontend/replay_scanners/ReplayScannersScene.js 90.1 KiB 🔺 +1.3 KiB (+1.5%)
posthog-app/src/lib/monaco/CodeEditorImpl.js 29.0 KiB 🔺 +1.2 KiB (+4.4%)
posthog-app/_parent/products/replay_vision/frontend/replay_scanners/ScannerEditorScene.js 61.9 KiB 🔺 +1.2 KiB (+2.0%)
posthog-app/src/scenes/project-homepage/ProjectHomepage.js 42.0 KiB 🔺 +1.2 KiB (+3.0%)
posthog-app/src/scenes/session-recordings/playlist/SessionRecordingsPlaylistScene.js 20.5 KiB 🔺 +1.2 KiB (+6.1%)
posthog-app/src/scenes/session-recordings/file-playback/SessionRecordingFilePlaybackScene.js 19.9 KiB 🔺 +1.2 KiB (+6.2%)
posthog-app/src/scenes/max/Max.js 29.4 KiB 🔺 +1.2 KiB (+4.1%)
posthog-app/_parent/products/business_knowledge/frontend/scenes/BusinessKnowledgeSettingsScene.js 16.5 KiB 🔺 +1.2 KiB (+7.6%)
posthog-app/_parent/products/conversations/frontend/scenes/settings/SupportSettingsScene.js 15.7 KiB 🔺 +1.2 KiB (+8.0%)
posthog-app/_parent/products/error_tracking/frontend/posthogAi/ErrorTrackingWidget.js 16.7 KiB 🔺 +1.2 KiB (+7.5%)
posthog-app/_parent/products/replay_vision/frontend/observations/ObservationRecording.js 17.0 KiB 🔺 +1.2 KiB (+7.4%)
posthog-app/src/scenes/session-recordings/detail/SessionRecordingDetail.js 17.3 KiB 🔺 +1.2 KiB (+7.2%)
posthog-app/src/scenes/session-recordings/player/modal/SessionPlayerModal.js 17.9 KiB 🔺 +1.2 KiB (+6.9%)
posthog-app/src/scenes/session-recordings/SessionRecordings.js 16.5 KiB 🔺 +1.2 KiB (+7.6%)
posthog-app/src/scenes/sessions/SessionProfileScene.js 30.2 KiB 🔺 +1.2 KiB (+4.0%)
posthog-app/src/scenes/settings/SettingsMap.js 15.6 KiB 🔺 +1.2 KiB (+8.0%)
posthog-app/_parent/products/customer_analytics/frontend/scenes/CustomerAnalyticsConfigurationScene/CustomerAnalyticsConfigurationScene.js 17.3 KiB 🔺 +1.2 KiB (+7.2%)
posthog-app/src/scenes/session-recordings/kiosk/SessionRecordingsKiosk.js 25.2 KiB 🔺 +1.2 KiB (+4.8%)
posthog-app/src/scenes/session-recordings/settings/SessionRecordingsSettingsScene.js 17.5 KiB 🔺 +1.2 KiB (+7.1%)
posthog-app/src/scenes/settings/SettingsScene.js 19.2 KiB 🔺 +1.2 KiB (+6.4%)
posthog-app/_parent/products/error_tracking/frontend/scenes/ErrorTrackingScene/ErrorTrackingScene.js 77.7 KiB 🔺 +1.2 KiB (+1.5%)
exporter/src/lib/monaco/CodeEditorImpl.js 28.8 KiB 🔺 +1.2 KiB (+4.2%)
posthog-app/_parent/products/engineering_analytics/frontend/scenes/WorkflowRunsScene.js 23.5 KiB 🔺 +1.2 KiB (+5.2%)
posthog-app/_parent/products/posthog_ai/frontend/scenes/TaskTracker/components/SidePanelRunnerImpl.js 17.4 KiB 🔺 +1.1 KiB (+7.0%)
posthog-app/src/scenes/surveys/Survey.js 13.2 KiB 🔺 +1.1 KiB (+9.4%)
posthog-app/_parent/products/metrics/frontend/MetricsScene.js 95.0 KiB 🔺 +1.1 KiB (+1.2%)
posthog-app/_parent/products/endpoints/frontend/EndpointScene.js 62.6 KiB 🔺 +1.1 KiB (+1.8%)
posthog-app/src/scenes/data-warehouse/editor/EditorScene.js 10.3 KiB 🔺 +1.1 KiB (+11.4%)
posthog-app/src/scenes/terminal/TerminalDock.js 1.0 KiB 🔺 +1.0 KiB (new)
posthog-app/src/scenes/product-tours/ProductTour.js 181.4 KiB 🔺 +1.0 KiB (+0.6%)
posthog-app/src/scenes/web-analytics/WebAnalyticsScene.js 227.8 KiB 🔺 +1.0 KiB (+0.5%)
posthog-app/src/scenes/experiments/Experiment.js 289.4 KiB 🔺 +1.0 KiB (+0.4%)
posthog-app/_parent/products/customer_analytics/frontend/scenes/CustomerAnalyticsAccountScene/CustomerAnalyticsAccountScene.js 23.5 KiB 🔺 +1.0 KiB (+4.5%)
posthog-app/src/scenes/experiments/SharedMetrics/SharedMetric.js 20.4 KiB 🔺 +1021 B (+5.1%)
posthog-app/src/scenes/dashboard/Dashboard.js 14.2 KiB 🔺 +1020 B (+7.5%)
exporter/_parent/products/posthog_ai/frontend/components/tool/ReadFileContent.js 1016 B 🔺 +1016 B (new)
posthog-app/src/scenes/insights/InsightScene.js 50.1 KiB 🔺 +1006 B (+2.0%)

Posted automatically by build-bundle-size-report · uncompressed bytes from dist-report

✅ Eager graph — within budget

How much code each root ships on the eager path — downloaded and parsed before the surface is interactive. Measured from the esbuild output chunks (post-tree-shake, static imports only); lazy import() / React.lazy chunks are not counted.

Root Eager (shipped) Δ vs base Budget
entry (logged-out pages, app bootstrap)
src/index.tsx		 1.57 MiB · 22 files 🔺 +9.6 KiB (+0.6%) █████████░ 85.1% of 1.84 MiB
logged-out boot: index + App + bootApp (preloaded by every page, including /login)
src/index.tsx + src/scenes/App.tsx + src/scenes/bootApp.ts		 3.57 MiB · 628 files 🔺 +13.3 KiB (+0.4%) █████████░ 88.5% of 4.03 MiB
authenticated shell (every logged-in page)
src/scenes/AuthenticatedShell.tsx		 7.50 MiB · 2,430 files 🟢 -926.5 KiB (-10.8%) ████████░░ 78.8% of 9.51 MiB

🟢 node_modules/monaco-editor/ stays out of src/index.tsx
🟢 src/lib/components/ActivityLog/describers stays out of src/index.tsx
🟢 [object Object] stays out of src/index.tsx
🟢 [object Object] stays out of src/index.tsx
🟢 node_modules/monaco-editor/ stays out of src/index.tsx + src/scenes/App.tsx + src/scenes/bootApp.ts
🟢 src/layout/navigation-3000/navigationLogic.tsx stays out of src/index.tsx + src/scenes/App.tsx + src/scenes/bootApp.ts
🟢 src/scenes/dashboard/dashboardLogic.tsx stays out of src/index.tsx + src/scenes/App.tsx + src/scenes/bootApp.ts
🟢 src/lib/lemon-ui/LemonMarkdown/ stays out of src/index.tsx + src/scenes/App.tsx + src/scenes/bootApp.ts
🟢 src/lib/components/RichContentEditor/ stays out of src/index.tsx + src/scenes/App.tsx + src/scenes/bootApp.ts
🟢 src/lib/components/CodeSnippet/ stays out of src/index.tsx + src/scenes/App.tsx + src/scenes/bootApp.ts
🟢 src/taxonomy/core-filter-definitions-by-group.json stays out of src/index.tsx + src/scenes/App.tsx + src/scenes/bootApp.ts
🟢 node_modules/monaco-editor/ stays out of src/scenes/AuthenticatedShell.tsx
🟢 src/lib/components/ActivityLog/describers stays out of src/scenes/AuthenticatedShell.tsx
🟢 [object Object] stays out of src/scenes/AuthenticatedShell.tsx
🟢 src/scenes/session-recordings/player/sessionRecordingPlayerLogic.ts stays out of src/scenes/AuthenticatedShell.tsx
🟢 [object Object] stays out of src/scenes/AuthenticatedShell.tsx
🟢 [object Object] stays out of src/scenes/AuthenticatedShell.tsx
🟢 [object Object] stays out of src/scenes/AuthenticatedShell.tsx

Largest files eagerly shipped from src/index.tsx
Size File
126.8 KiB ../node_modules/.pnpm/react-dom@18.3.1_react@18.3.1/node_modules/react-dom/cjs/react-dom.production.min.js
24.6 KiB ../node_modules/.pnpm/buffer@6.0.3/node_modules/buffer/index.js
6.3 KiB ../node_modules/.pnpm/react@18.3.1/node_modules/react/cjs/react.production.min.js
4.5 KiB ../node_modules/.pnpm/@jspm+core@2.1.0/node_modules/@jspm/core/nodelibs/browser/process.js
3.9 KiB ../node_modules/.pnpm/scheduler@0.23.2/node_modules/scheduler/cjs/scheduler.production.min.js
1.4 KiB ../node_modules/.pnpm/base64-js@1.5.1/node_modules/base64-js/index.js
1.3 KiB src/index.tsx
1.3 KiB src/RootErrorBoundary.tsx
912 B ../node_modules/.pnpm/ieee754@1.2.1/node_modules/ieee754/index.js
854 B src/scenes/ChunkLoadErrorBoundary.tsx
Largest files eagerly shipped from src/index.tsx + src/scenes/App.tsx + src/scenes/bootApp.ts
Size File
301.8 KiB ../node_modules/.pnpm/posthog-js@1.434.11_@types+react@18.3.27_react@18.3.1/node_modules/posthog-js/dist/module.mjs
267.6 KiB ../node_modules/.pnpm/@posthog+icons@0.38.0_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/@posthog/icons/dist/posthog-icons.es.js
126.8 KiB ../node_modules/.pnpm/react-dom@18.3.1_react@18.3.1/node_modules/react-dom/cjs/react-dom.production.min.js
100.4 KiB src/lib/api.ts
83.9 KiB src/products.tsx
69.1 KiB src/lib/lemon-ui/icons/icons.tsx
63.4 KiB src/lib/utils/eventUsageLogic.ts
38.7 KiB ../node_modules/.pnpm/@dnd-kit+core@6.0.8_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/@dnd-kit/core/dist/core.esm.js
33.9 KiB ../node_modules/.pnpm/kea@4.0.0-pre.6_patch_hash=139b8d1f1304f9d9da452a9a1244c94ea679dbcb85687d8999563146879fb6f5_react@18.3.1/node_modules/kea/lib/index.cjs.js
28.5 KiB src/scenes/scenes.ts
Largest files eagerly shipped from src/scenes/AuthenticatedShell.tsx
Size File
301.8 KiB ../node_modules/.pnpm/posthog-js@1.434.11_@types+react@18.3.27_react@18.3.1/node_modules/posthog-js/dist/module.mjs
268.0 KiB src/taxonomy/core-filter-definitions-by-group.json
267.6 KiB ../node_modules/.pnpm/@posthog+icons@0.38.0_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/@posthog/icons/dist/posthog-icons.es.js
153.7 KiB ../node_modules/.pnpm/re2js@0.4.1/node_modules/re2js/build/index.esm.js
126.8 KiB ../node_modules/.pnpm/react-dom@18.3.1_react@18.3.1/node_modules/react-dom/cjs/react-dom.production.min.js
100.4 KiB src/lib/api.ts
98.5 KiB ../packages/quill/packages/quill/dist/index.js
93.3 KiB ../node_modules/.pnpm/prosemirror-view@1.40.1/node_modules/prosemirror-view/dist/index.js
90.6 KiB ../node_modules/.pnpm/@tiptap+core@3.20.6_@tiptap+pm@3.20.6/node_modules/@tiptap/core/dist/index.js
83.9 KiB src/products.tsx

Posted automatically by check-eager-graph · sizes are eager output bytes (shipped, post-tree-shake) from the esbuild metafile · part of #32479

✅ Toolbar bundle — eager 2.36 MiB within budget

What the toolbar ships to customer pages, measured from the esbuild output (minified, post-tree-shake). The eager set is the entry plus everything statically imported from it — fetched before any feature runs; deferred chunks load lazily. The eager guardrail is 5.72 MiB. Each output file must also stay below 10 MB, where CloudFront stops compressing it. The module boundary is enforced separately by check-toolbar-graph.

Metric Size Δ vs base Budget
Eager (shipped)
entry + static imports		 2.36 MiB · 19 files 🔺 +6.1 KiB (+0.3%) ████░░░░░░ 41.3% of 5.72 MiB
Deferred (lazy) 2.09 MiB · 44 files 🔺 +28 B (+0.0%) n/a — loads on demand
Loader dist/toolbar.js 1.2 KiB no change █░░░░░░░░░ 6.0% of 19.5 KiB
Largest eagerly-shipped chunks
Size File
787.1 KiB dist/toolbar/toolbar-app-IBRAWU24.css
650.5 KiB dist/toolbar/chunk-chunk-ADLZ7WB6.js
483.6 KiB dist/toolbar/chunk-chunk-EC4CHVWG.js
138.3 KiB dist/toolbar/chunk-chunk-YY3RT3CP.js
131.8 KiB dist/toolbar/chunk-chunk-FDH2IBXT.js
75.2 KiB dist/toolbar/toolbar-app-KJEU4DM5.js
69.0 KiB dist/toolbar/chunk-chunk-TSAL54PB.js
35.6 KiB dist/toolbar/chunk-chunk-ZH2PRIEQ.js
21.0 KiB dist/toolbar/chunk-chunk-FSNY4LOM.js
6.8 KiB dist/toolbar/chunk-chunk-DV7IWQNF.js

Posted automatically by check-toolbar-size · sizes are toolbar output bytes (shipped, post-tree-shake) from the esbuild metafile

⚠️ Dist folder size — 🔺 +32.38 MiB (+2.1%)

Total size of the built frontend/dist folder (all assets), compared against the base branch.

Total: 1543.25 MiB · 🔺 +32.38 MiB (+2.1%)

@trunk-io

trunk-io Bot commented Sep 22, 2026 •
edited
Loading

Copy link
Copy Markdown

Static Badge   Static Badge   Static Badge

View Full Report ↗︎ ⋅ Docs

@pauldambra
pauldambra marked this pull request as ready for review September 24, 2026 11:08
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 24, 2026 •
edited
Loading

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-24T11:11:32.542981Z 8ca6b6a Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@pauldambra pauldambra added the stamphog Request AI approval (no full review) label Sep 24, 2026 — with PostHog
@pr-assigner-resolver-posthog
pr-assigner-resolver-posthog Bot requested a review from a team September 24, 2026 11:09
stamphog[bot]
stamphog Bot reviewed Sep 24, 2026
View reviewed changes

@stamphog stamphog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not approved yet — waiting on the conditions below.

Re-add the stamphog label to request another review once you have addressed this.

Gates denied this PR because it edits a CI workflow file (deny-listed infra/CI territory) and is classified as never-auto-approvable, regardless of how small or well-reasoned the change looks; it needs a human to sign off.

  • webjunkie reviewed the current head.
  • Modifies .github/workflows/ci-frontend.yml, which is on the CI/infra deny-list and gated to always require human review regardless of size or existing approval.
Gate mechanics and policy version
Gate Result
prerequisites ✓ all clear
deny-list ✗ matches: infra_cicd
size ✓ 2L, 1F substantive — within ceiling
tier ✗ classified as T2-never: T2-never (2L, 1F, single-area, fix)
stamphog 2.0.0 .stamphog/policy.yml @ 8ca6b6a · reviewed head 8ca6b6a

@stamphog stamphog Bot removed the stamphog Request AI approval (no full review) label Sep 24, 2026
@pauldambra

pauldambra commented Sep 24, 2026

Copy link
Copy Markdown
Member Author

Note

🤖 Automated comment by QA Swarm — not written by a human

Multi-perspective review: router (cheap-first pass) + delegated reviewers (qa-team, paul-reviewer, xp-reviewer, security-audit as warranted)

Verdict: ✅ APPROVE (round 1 @ 8ca6b6a)

Small, low-risk CI workflow change. The router traced the full test-verdict flow (Jest run → JUnit files → quarantine gate → job exit) and confirmed the artifact upload step is purely archival: the quarantine gate reads local JUnit files and keys off steps.run-jest.outcome, so continue-on-error: true on the upload step cannot mask a real test failure.

Key findings

None.

Convergence

N/A — single-pass review, no delegation triggered (danger: LOW, confidence: HIGH).

Reviewer summaries

Reviewer Assessment
🧭 router LOW danger, HIGH confidence. Verified the upload step is independent of the test-verdict path, YAML is valid, and the change matches existing continue-on-error conventions elsewhere in the same job. No delegation needed.

Automated by QA Swarm — not a human review

@pauldambra pauldambra added the stamphog Request AI approval (no full review) label Sep 24, 2026
stamphog[bot]
stamphog Bot reviewed Sep 24, 2026
View reviewed changes

@stamphog stamphog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not approved yet — waiting on the conditions below.

Re-add the stamphog label to request another review once you have addressed this.

This pull request was refused automatically by stamphog's deterministic gates, not reviewed for content.

  • deny-list: FAILED — matches infra_cicd, since this change modifies .github/workflows/ci-frontend.yml, a CI/infra workflow file that this policy category always blocks from automated handling.
  • tier: FAILED — classified as T2-never, meaning this kind of change (small, single-area fix to CI workflow config) is never eligible for automated approval regardless of size.

Both gate failures are final and independent of code quality. To proceed, please request a human reviewer for this pull request; if it can be broken into smaller or differently scoped changes that avoid touching CI/infra workflow files, that may also help with future automated passes.

  • webjunkie reviewed the current head.
Gate mechanics and policy version
Gate Result
prerequisites ✓ all clear
deny-list ✗ matches: infra_cicd
size ✓ 2L, 1F substantive — within ceiling
tier ✗ classified as T2-never: T2-never (2L, 1F, single-area, fix)
stamphog 2.1.0 .stamphog/policy.yml @ unknown · reviewed head 8ca6b6a

@stamphog stamphog Bot removed the stamphog Request AI approval (no full review) label Sep 24, 2026
@posthog posthog Bot mentioned this pull request Sep 25, 2026
Merged
1 of 4 tasks
@trunk-io
trunk-io Bot merged commit 317ea09 into master Sep 25, 2026
412 checks passed
@trunk-io
trunk-io Bot deleted the posthog/fix-frontend-artifact-upload branch September 25, 2026 00:20
@deployment-status-posthog

deployment-status-posthog Bot commented Sep 25, 2026 •
edited
Loading

Copy link
Copy Markdown

Deploy status

Environment Status Deployed At Workflow
dev ✅ Deployed 2026-09-25 00:51 UTC Run
prod-us ✅ Deployed 2026-09-25 01:02 UTC Run
prod-eu ✅ Deployed 2026-09-25 01:03 UTC Run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stamphog stamphog[bot] stamphog[bot] left review comments
@webjunkie webjunkie webjunkie approved these changes

Assignees

@pauldambra pauldambra

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pauldambra @webjunkie